Skip to content

Conversation

shruthilayaj
Copy link
Member

It's already prevented in the UI when the flag is enabled

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 8, 2025
}
mock_find_channel_id_for_alert_rule.assert_called_once_with(kwargs=kwargs)

def test_eap_alert_with_invalid_time_window(self) -> None:
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just moved this test to the right class

Copy link

codecov bot commented Oct 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #101177      +/-   ##
===========================================
- Coverage   81.09%    81.09%   -0.01%     
===========================================
  Files        8669      8669              
  Lines      384582    384607      +25     
  Branches    24275     24275              
===========================================
+ Hits       311881    311896      +15     
- Misses      72354     72364      +10     
  Partials      347       347              

Copy link
Member

@ceorourke ceorourke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks shruthi!

@shruthilayaj shruthilayaj merged commit c7a6710 into master Oct 8, 2025
66 checks passed
@shruthilayaj shruthilayaj deleted the shruthi/chore/prevent-users-from-creating-txn-alerts-api branch October 8, 2025 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants